Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update links to Universal examples in README.md #614

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

lurch
Copy link
Contributor

@lurch lurch commented Feb 19, 2025

The universal/CMakeLists.txt has been edited, so the line-numbers had drifted

The `universal/CMakeLists.txt` has been edited, so the line-numbers had drifted
@@ -386,9 +386,9 @@ These require you to set `PICO_ARM_TOOLCHAIN_PATH` and `PICO_RISCV_TOOLCHAIN_PAT

App|Description
---|---
[blink](universal/CMakeLists.txt#L126) | Same as the [blink](blink) example, but universal.
[blink](universal/CMakeLists.txt#L137) | Same as the [blink](blink) example, but universal.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are the line numbers there at all?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to git blame, the line-numbers first appeared in 7fe60d6
Ping @will-v-pi and @kilograham

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It’s because those examples don’t have their own folders, they’re just created by lines in the CMakeLists.txt file. The actual example code is in the normal blink/nuke folder.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants